Auto merge of #1310 - tanadeau:use-struct-variant, r=alexcrichton

Minor code cleanup for readability and usability for `Method` enum.

Auto merge of #1301 - brson:installer-next, r=alexcrichton

Needs to land immediately after https://github.com/rust-lang/rust/pull/22256.

Use struct variant instead of long, commented tuple

Auto merge of #1284 - apasel422:issue-1277, r=alexcrichton

This is a WIP for #1277. I would like to add a test for the new behavior in `tests/test_cargo_test.rs`, but I'm having a hard time getting the output to match.

Some feedback would be great to see if this is even the right approach.

Auto merge of #1292 - brson:version, r=huonw

```
cargo 0.0.1-pre (9404539 2015-02-09 20:54:26 +0000) (built 2015-02-11)
```

Add the build date to the reported version

```
cargo 0.0.1-pre (9404539 2015-02-09) (built 2015-02-11)
```

pass features to doctest binary

Upgrade rust-installer

update Rust

Auto merge of #1283 - alexcrichton:bad-error, r=huonw

Closes rust-lang/crates.io#130

Auto merge of #1285 - alexcrichton:issue-636, r=huonw

Due to libgit2 not supporting HTTP proxies, the custom transport API of the
library must be used to reimplement the HTTP transport with proxy support. The
git2-curl crate implements this transport on top of the curl-rust crate. By
using libcurl we gain all sorts of proxy support for free.

This transport is not used by default, however, as it is not well battle tested
and the architecture is not currently ideal (download the entire repo into
memory on a clone). Only when an HTTP proxy is present is the new transport
used.

The other drawback of git2-curl is that it does not currently support
authentication. If a private git repository is cloned or authentication is
required then it will generate an error instead of correctly asking for
credentials.

Closes #636

Add git2-curl as a dependency

Due to libgit2 not supporting HTTP proxies, the custom transport API of the
library must be used to reimplement the HTTP transport with proxy support. The
git2-curl crate implements this transport on top of the curl-rust crate. By
using libcurl we gain all sorts of proxy support for free.

This transport is not used by default, however, as it is not well battle tested
and the architecture is not currently ideal (download the entire repo into
memory on a clone). Only when an HTTP proxy is present is the new transport
used.

The other drawback of git2-curl is that it does not currently support
authentication. If a private git repository is cloned or authentication is
required then it will generate an error instead of correctly asking for
credentials.

Closes #636

Auto merge of #1278 - alexcrichton:style, r=brson

* Update to rust master
* Remove many calls to `.as_slice` (deref coercions)
* Remove `get_` prefixes

Update to rust master

Auto merge of #1282 - vrinek:master, r=alexcrichton

Improve error message for missing example/bin

Closes rust-lang/crates.io#130

Add link to requirement strings explanation in the manifest.md doc

Remove leading `get_` prefix from accessors

The current naming is [against our guidelines][guidelines]

[guidelines]: https://github.com/rust-lang/rfcs/blob/master/text/0344-conventions-galore.md#gettersetter-apis

Down with .as_slice()!

Update to rust master

Auto merge of #1271 - alexcrichton:issue-1267, r=brson

Why not use `try!` instead!

Closes #1267

Auto merge of #1270 - alexcrichton:issue-1105, r=brson

Closes #1105

Auto merge of #1269 - alexcrichton:issue-961, r=brson

Over time this functionality has become obsolete through other means. Due to the
usage of `-L dependency=foo` it's not possible to pick up stale dependencies by
accident, and due to `--extern` you can only pick up a dependency. All of the
cases that auto-cleaning was fixing are now fixed through other methods, so
there's not much use ensuring a "clean build directory" any more.

This has the benefit of fixing issues like #961 where the downside of long
compiles outweighs the benefits of a "let's pretend we started from scratch"
build.

Closes #961

Auto merge of #1261 - alexcrichton:issue-1259, r=wycats

Previously if a fingerprint was considered fresh based on mtime, it would not
get updated once a compilation finished with the new precise fingerprint (it
would use the older fingerprint). This alters the `resolve` method to take a
flag which disables this behavior and forces looking at the filesystem for a
fingerprint.

Closes #1259

Add a flag to force resolution of a fingerprint

Previously if a fingerprint was considered fresh based on mtime, it would not
get updated once a compilation finished with the new precise fingerprint (it
would use the older fingerprint). This alters the `resolve` method to take a
flag which disables this behavior and forces looking at the filesystem for a
fingerprint.

Closes #1259

Remove auto-cleaning of the build directory

Over time this functionality has become obsolete through other means. Due to the
usage of `-L dependency=foo` it's not possible to pick up stale dependencies by
accident, and due to `--extern` you can only pick up a dependency. All of the
cases that auto-cleaning was fixing are now fixed through other methods, so
there's not much use ensuring a "clean build directory" any more.

This has the benefit of fixing issues like #961 where the downside of long
compiles outweighs the benefits of a "let's pretend we started from scratch"
build.

Closes #961

Auto merge of #1275 - alexcrichton:update, r=alexcrichton

Expanding on the work of #1256 to update to rustc 02-02

Update to the most current nightly

Merge branch 'rust-master' of https://github.com/gbsf/cargo into update

Avoid `unwrap` when parsing lockfiles

Why not use `try!` instead!

Closes #1267

Support update --precise for registry deps

Closes #1105

Squelch warnings

Fix failing test

Remove unnecessary allow(unstable)

Update rustversion.txt

Workaround to make code compile

Blame @alexcrichton

Update dependencies

Update log crate to v0.2

Writer::write → Writer::write_all

std::io → std::old_io

Auto merge of #1250 - alexcrichton:issue-1248, r=brson

When probing a repository for files that should be considered as inputs for a
build script we should just skip submodules that haven't been checked out
instead of throwing an error.

Closes #1248

Auto merge of #1243 - alexcrichton:issue-1239, r=brson

Previously a build script would assign itself to the platform requirement for
the first target of a package, but it's more correct to consider all targets for
platform requirements as there may be both a plugin and target requirement.

Closes #1236

Auto merge of #1247 - tshepang:hgignore-love, r=alexcrichton

Auto merge of #1242 - alexcrichton:issue-1236, r=wycats

Set out to solve #1236, but ended up solving #834 as well!

Don't require all submodules are checked out

When probing a repository for files that should be considered as inputs for a
build script we should just skip submodules that haven't been checked out
instead of throwing an error.

Closes #1248

Forbid cyclic dependencies

Re-jigger some code in the resolver and refactor a little bit to get the
ordering of checks right to forbid cyclic dependencies from ever reaching the
backend.

Closes #834

Consider all targets for platform requirements

Previously a build script would assign itself to the platform requirement for
the first target of a package, but it's more correct to consider all targets for
platform requirements as there may be both a plugin and target requirement.

Closes #1230

new: use .{git,hg}ignore syntax that's also understood by Mercurial

Consider transitive fingerprints for freshness

Originally discovered through #1236, this commit fixes a bug in Cargo where
crates may not be recompiled when they need to (leading to obscure errors from
the compiler). The scenario in question looks like:

* Assume a dependency graph of `A -> B -> C` and `A -> C`
* Build all packages
* Modify C
* Rebuild, but hit Ctrl+C while B is building
* Modify A
* Rebuild again

Previously, Cargo only considered the freshness of a package to be the freshness
of the package itself (checking source files, for example). To handle transitive
recompilations, Cargo propagates a dirty bit throughout the dependency graph
automatically (instead if calculating it as such).

In the above example, however, we have a problem where as part of the last
rebuild Cargo thinks `B` and `C` are fresh! The artifact for `C` was just
recompiled, but `B`'s source code is untainted, so Cargo does not think that it
needs to recompile `B`. This is wrong, however, because one of `B`'s
dependencies was rebuilt, so it needs to be rebuilt.

To fix this problem, the fingerprint (a short hash) for all packages is now
transitively propagated (the fingerprint changes when an upstream package
changes). This should ensure that even when Ctrl+C is hit (or the situation
explained in #1236) that Cargo will still consider packages whose source code is
untainted as candidates for recompilation.

The implementation is somewhat tricky due to the actual fingerprint for a path
dependency not being known until *after* the crate is compiled (the fingerprint
is the mtime of the dep-info file).

Closes #1236

Auto merge of #1232 - alexcrichton:issue-1231, r=huonw

* Break out the error reporting to its own separate function
* Don't pass `Context` by value as it's quite large, instead pass `Box<Context>`
* Don't bother monomorphizing `R: Registry`, instead just take a trait object.

Closes #1231

Auto merge of #1238 - SimonSapin:patch-2, r=alexcrichton

Update a code example for language change

Optimize stack usage of activate_deps slightly

* Break out the error reporting to its own separate function
* Don't pass `Context` by value as it's quite large, instead pass `Box<Context>`
* Don't bother monomorphizing `R: Registry`, instead just take a trait object.

Closes #1231

Auto merge of #1227 - retep998:master, r=huonw

Fixes the following issue while building Cargo
```
   Compiling cargo v0.1.0 (file:///C:/msys64/home/Peter/cargo)
src\cargo\lib.rs:7:1: 7:20 error: can't find crate for `regex`
src\cargo\lib.rs:7 extern crate regex;
                   ^~~~~~~~~~~~~~~~~~~
error: aborting due to previous error
Could not compile `cargo`.
```

Add explicit dependency on regex.

Signed-off-by: Peter Atashian <retep998@gmail.com>

Auto merge of #1222 - alexcrichton:update, r=alexcrichton

Update to rust 2015-01-24

Merge branch 'master' of https://github.com/suhr/cargo into update

Auto merge of #1218 - emanueLczirai:verbose1, r=alexcrichton

and also show that cargo's version after rustc's one

Closes #1216

pass --verbose to cargo when VERBOSE env is set

and also show that cargo's version after rustc's one

update to recent rust changes: part #2

fix build on recent rust: use `unwrap` on `encode`

update to recent rust changes: part #1

Auto merge of #1202 - alexcrichton:issue-1201, r=huonw

Previously an error's description was not printed at all if it was not a "human
error". This alters the logic to always print the description on --verbose, as
well as ensuring that --verbose is suggested if this is the case and we're not
verbose.

Closes #1201

Auto merge of #1205 - alexcrichton:case-sensitivity, r=brson

All crates published will enter the index in an all-lowercase fashion. This is
done because the index may be checked out on case-insensitive filesystems and
its integrity needs to be preserved. When looking for a file in the index, a
crate's name is transformed to all lowercase, but the mentioned name in the
index is verified against the name queried to ensure that they match.

Closes rust-lang/crates.io#93
cc rust-lang/crates.io-index#1

Search the index case-insensitively

All crates published will enter the index in an all-lowercase fashion. This is
done because the index may be checked out on case-insensitive filesystems and
its integrity needs to be preserved. When looking for a file in the index, a
crate's name is transformed to all lowercase, but the mentioned name in the
index is verified against the name queried to ensure that they match.

Closes rust-lang/crates.io#93
cc rust-lang/crates.io-index#1

Auto merge of #1199 - retep998:winapi, r=alexcrichton

Fix printing full descriptions on --verbose

Previously an error's description was not printed at all if it was not a "human
error". This alters the logic to always print the description on --verbose, as
well as ensuring that --verbose is suggested if this is the case and we're not
verbose.

Closes #1201

Use external crates for Windows FFI

Signed-off-by: Peter Atashian <retep998@gmail.com>

Auto merge of #1190 - alexcrichton:issue-1186, r=alexcrichton

Closes #1186

Auto merge of #1195 - alexcrichton:no-more-build-command, r=huonw

Along the way this also fixes a few bugs with build scripts with respect to dynamic native libraries which are linked into build scripts themselves.

Auto merge of #1196 - alexcrichton:issue-1187, r=alexcrichton

Closes #1187

Fix code example in the guide

Closes #1187

Augment dylib search paths for rustc/build scripts

Whenever a build script or a plugin depends on a native dynamic library, the
dylib search path will be required to include the path to the generated dynamic
library. Currently Cargo does not augment the search path, causing load errors
or execution errors whenever builds are attempted in this case.

This commit remedies the situation by ensuring that dynamic libraries show up in
dylib search paths. This commit is necessary to fully migrate the remaining
tests from the old build command infrastructure to build scripts.

Remove infrastructure for the old custom build command

This aspect of the manifest has been deprecated for quite some time now, and
this commit purges the support entirely.

Auto merge of #1193 - alexcrichton:update, r=huonw

One important update this picks up is toml-rs v0.1.13 which includes some recent changes to the TOML spec (shouldn't be breaking for Cargo though)

Update to rust master

Auto merge of #1191 - alexcrichton:fix-bots, r=huonw

The nightly bots are all failing because a nightly build emits a .cargo/config
which uses `[target]` for some overrides, but a test was creating invalid
configuration for `[target]`, causing the test error message to change.

This commit fixes the test by using a different error message (which should not
conflict). It also improves the error message for "configuration could not be
merged" to include more contextual information about what just happened.

Fix nightly and improve merge error message

The nightly bots are all failing because a nightly build emits a .cargo/config
which uses `[target]` for some overrides, but a test was creating invalid
configuration for `[target]`, causing the test error message to change.

This commit fixes the test by using a different error message (which should not
conflict). It also improves the error message for "configuration could not be
merged" to include more contextual information about what just happened.

Update `cargo update` syntax in the guide

Closes #1186

Auto merge of #1189 - alexcrichton:issue-1185, r=huonw

Closes #1185

Update build script docs for rustc changes

Closes #1185

Auto merge of #1183 - parkr:patch-1, r=wycats

This seemed a little crucial.

Add note about specifying dependencies.

Auto merge of #1176 - alexcrichton:less-relative, r=alexcrichton

Turns out path_relative_from returns Some even if they're not ancestors of one
another! This adds an explicit check for `is_ancestor_of`.

Actually use absolute paths for crates.io crates

Turns out path_relative_from returns Some even if they're not ancestors of one
another! This adds an explicit check for `is_ancestor_of`.

Auto merge of #1172 - alexcrichton:issue-1087, r=huonw

These commits aim to streamline how errors are generated and how configuration is read to be less error prone and more easily extensible.

This was a whole bunch of yak shaving when tackling #1087.

Print human errors in traces by default

This helps get nice pretty stack traces which terminate when we get into the
weeds of errors but still present relevant contextual information when Cargo
isn't run with --verbose

Don't lose error traces in manifest parse errors

Closes #1174

Allow configuration of jobs by default via build.jobs

Continuation of #1114
Closes #1108

Add a configuration option for HTTP timeouts

Closes #1087

Vastly improve configuration error messages

This commit seeks to improve error messages with respect to a misconfigured
Cargo. This is done with a few principles and architectural changes:

* All error messages related to configuration should now mention the file in
  question.
* All error messages should print the key in question which was in error.
* Loading configuration values is now centralized in the `Config` structure,
  providing precisely one location where error message originate from. The
  messages were previously strewn and duplicated about.
* The `Config` structure now provides typed `get_foo` methods to load various
  configuration keys of various types.

Overhaul how Config is passed around

This commit started out by moving the hash map of configuration values into
`Config` and then removing all calls to config::all_configs in favor of taking a
`&Config` argument. This initial step was taken to prevent reloading
configuration many times from disk, but it also provides a nice house for
configuration to be loaded from.

The commit snowballed into purging `MultiShell` as an argument all over the
place and passing around `Config` instead. Along the way a few other breaking
changes were made:

* The `config_for_key` and `config_list` subcommands have been removed. These
  commands are not ready to be stabilized and need some more work. In general
  the `cargo config` system needs to be cleaned out and built back up with a new
  interface, at which point these subcommands can return.
* The old and deprecated behavior of `cargo update foo` has been removed.

Auto merge of #1166 - alexcrichton:docs, r=steveklabnik

Add a short section about path dependencies

Closes #640

Call out libs a little more in the guide

Closes #643

Clarify that features are actually sets of features

Closes #938

Mention profiles in --help pages

Closes #999

Auto merge of #1151 - alexcrichton:issue-1145, r=wycats

The new message will print out a selection of versions that were found if any
are to be had, as well as a recommendation to run `cargo update` if it's a path
dependency.

Closes #1145

Print nicer messages on 'no package named' errors

The new message will print out a selection of versions that were found if any
are to be had, as well as a recommendation to run `cargo update` if it's a path
dependency.

Closes #1145